-
Notifications
You must be signed in to change notification settings - Fork 12
Antalya-25.8 - Move workflows and docs #1300
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: antalya-25.8
Are you sure you want to change the base?
Conversation
6646a8a to
828e7e7
Compare
| repo-sanity-check: | ||
| needs: sign-and-publish | ||
| uses: Altinity/ClickHouse/.github/workflows/repo-sanity-checks.yml@antalya |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ref should match whatever our main branch is. I'm not sure if there is a way to automate this, there is probably some github var that shows current branch, but the ref can't be left empty.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ref is only needed if sign_and_release.yml is run from a non-main branch. Do we expect that to occur?
| repository: Altinity/ClickHouse | ||
| ref: antalya | ||
| path: ClickHouse | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't let me add a comment to this line specifically, but the ref in the checkout can be removed.
- name: Checkout repository
uses: actions/checkout@v4
with:
repository: Altinity/ClickHouse
ref: antalya
path: ClickHouse
Changelog category (leave one):
TODO
.github/workflows/README.mdwas excluded due to being outdatedCI/CD Options
Exclude tests:
Regression jobs to run: